Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add API to return list of downstream contexts for an upstream #739

Conversation

rpenido
Copy link
Member

@rpenido rpenido commented Feb 14, 2025

@rpenido rpenido force-pushed the rpenido/fal-4005/list-courses-using-library branch 3 times, most recently from d4ad01b to 4516aa2 Compare February 14, 2025 04:08
@@ -29,6 +29,11 @@
downstreams.UpstreamListView.as_view(),
name='upstream-list'
),
re_path(
f'^upstream/{settings.USAGE_KEY_PATTERN}/downstream-contexts$',
Copy link
Member Author

@rpenido rpenido Feb 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not happy with my URL choice here. I was trying to have a structure like parent/{parentId}/children instead of children/{parentId}, but I`m not sure if this is clear close to the other endpoints.

Do you think there is a better way to make it consistent with the other URLs?

@rpenido rpenido force-pushed the rpenido/fal-4005/list-courses-using-library branch from 4516aa2 to 88e9b08 Compare February 14, 2025 04:11
feanil and others added 12 commits February 14, 2025 14:52
These app models get pulled in when trying to build the OpenAPI docs so
we need their relevant apps installed into the environment used to
generate the docs.
The format of the `source_suffix` setting has change but the default is
fine for us so we don't need to override this.
Adds api for listing upstream entity links for a given course.
chore: Update xblocks-contrib pypi package

Commit generated by workflow `openedx/edx-platform/.github/workflows/upgrade-one-python-dependency.yml@refs/heads/master`
…contrib-a1d0826

feat: Upgrade Python dependency xblocks-contrib
…6199)

* fix: Render Word cloud and conditional block editor

- The xmodule-type to render is MetadataOnlyEditingDescriptor
- The xmodule type `MetadataOnlyEditingDescriptor` renders a `<div>` with the block metadata in the `data-metadata` attribute. But is necessary to call `XBlockEditorView.xblockReady()` to run the scripts to build the editor using the metadata.
- To call XBlockEditorView.xblockReady() we need a specific require.config

* fix: Adding save and cancel button

* fix: save with studio_submit of conditional_block and word_cloud_block

* test: Tests for studio_submit of conditional and word cloud

* revert: Delete studio_submit of conditional block. It is not supported

* style: Fix lint

---------

Co-authored-by: Navin Karkera <[email protected]>
@rpenido rpenido force-pushed the rpenido/fal-4005/list-courses-using-library branch from 5ca5d06 to 5d21ba3 Compare February 19, 2025 14:57
@rpenido rpenido force-pushed the rpenido/fal-4005/list-courses-using-library branch 5 times, most recently from c752f6d to e92aa0f Compare February 20, 2025 02:42
@rpenido rpenido force-pushed the rpenido/fal-4005/list-courses-using-library branch from e92aa0f to c8bbff9 Compare February 20, 2025 02:56
@rpenido rpenido closed this Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants